Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

When binding roles, the user list is changed to the drop-down selection mode (在绑定角色的时候用户列表改成下拉选中的模式) #3518 #3522

Merged
merged 17 commits into from
Aug 6, 2020

Conversation

ZZQ001010
Copy link
Contributor

在绑定角色的时候用户列表改成下拉选中的模式 #3518

@ZZQ001010 ZZQ001010 changed the title 在绑定角色的时候用户列表改成下拉选中的模式 #3518 When binding roles, the user list is changed to the drop-down selection mode (在绑定角色的时候用户列表改成下拉选中的模式) #3518 Aug 4, 2020
@KomachiSion
Copy link
Collaborator

Please use nacos code style to reformat your change of java code
detail see src/style/codeStyle.md

@@ -39,13 +40,13 @@
@Conditional(value = ConditionOnEmbeddedStorage.class)
@Component
public class EmbeddedUserPersistServiceImpl implements UserPersistService {

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do not change indent please.
please check for all your changed files

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Github show the indent still be changed

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am changing

@@ -39,13 +40,13 @@
@Conditional(value = ConditionOnEmbeddedStorage.class)
@Component
public class EmbeddedUserPersistServiceImpl implements UserPersistService {

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Github show the indent still be changed

@ZZQ001010
Copy link
Contributor Author

@ready to close@


@Override
public List<User> findUserLikeUsername(String username) {
String sql = "SELECT username FROM users WHERE username like '%' ? '%'";
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the SQL is right? Can you make sure this SQL can run both in derby and mysql?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the SQL will be handle to SELECT username FROM users WHERE username like '%' 'xxx' '%'; if username is xxx


@Override
public List<User> findUserLikeUsername(String username) {
String sql = "SELECT username,password FROM users WHERE username like '%' ? '%'";
Copy link
Collaborator

@KomachiSion KomachiSion Aug 5, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only query username in Embedded, but username,password in Extranal?

@ZZQ001010
Copy link
Contributor Author

@KomachiSion Don't close this ,I will optimize similar functions in other places tomorrow
不要关闭这个issues ,我明天把其他页面相似的功能加上

@KomachiSion
Copy link
Collaborator

@KomachiSion Don't close this ,I will optimize similar functions in other places tomorrow
不要关闭这个issues ,我明天把其他页面相似的功能加上

One PR one change, If you want to do some change about other feature, please open new PR.

@KomachiSion KomachiSion added area/Nacos console Related to Nacos consle kind/enhancement Category issues or prs related to enhancement. kind/user experience not necessarily an error but can be improved for user experience labels Aug 6, 2020
@KomachiSion KomachiSion added this to the 1.4.0 milestone Aug 6, 2020
@KomachiSion KomachiSion merged commit 7a471c5 into alibaba:develop Aug 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/Nacos console Related to Nacos consle kind/enhancement Category issues or prs related to enhancement. kind/user experience not necessarily an error but can be improved for user experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants